Lists and keys by zahrajoulaei · Pull Request #36 · reactjs/fa.react.dev

zahrajoulaei · 2019-03-17T21:38:28Z

No description provided.

netlify · 2019-03-17T21:44:33Z

Deploy preview for fa-reactjs ready!

Built with commit 322d298

https://deploy-preview-36--fa-reactjs.netlify.com

sorousht · 2019-03-18T10:10:27Z

Hey @zahrajoulaei
Great! I will add my review by tonight.

sorousht · 2019-03-18T20:30:47Z

@zahrajoulaei عزیز

پیش از بررسی دقیق، مواردی به چشم می‌خورد که باید برطرف شود:
۱. رعایت نیم‌فاصله: در مواردی مانند: ری‌اکت، می‌شود، لیست‌ها و ... نیم فاصله رعایت نشده‌است.
۲. بعد از کاما (،) و کلمه بعد از آن یک فاصله باشد

sorousht · 2019-03-24T19:10:53Z

@zahrajoulaei عزیز، نتیجه بررسی نیمه اول این صفحه رو نوشتم. در دو روز آینده بخش دوم رو هم بررسی خواهم کرد. یکم گرفتار هستم و ببخشید اگر زمان زیادی برد.

zahrajoulaei · 2019-04-06T16:32:58Z

@sorousht همه ی تغییرات ‍‍‍پیشنهادی اعمال شد.مرسی از بررسی دقیق.

sorousht

@zahrajoulaei I found a couple of issues that might need to be improved.

sorousht · 2019-04-08T10:33:58Z

@zahrajoulaei Thanks for applying the changes. It's fine, but I do not merge it, because @sJJdGG has requested to review it.

zahrajoulaei · 2019-04-08T16:25:59Z

@zahrajoulaei Thanks for applying the changes. It's fine, but I do not merge it, because @sJJdGG has requested to review it.

Then we are waiting for @sJJdGG to merge this PR, but until when? she/he has requested to review it 12 days ago! I will not start working on another page until this one is merged.

sJJdGG

@zahrajoulaei @sorousht

ممنون بابت این PR. چند تا مورد بود که سراسری بود و خواستم نظرتون رو بدونم:

- استفاده از اعداد به جای numbers. به نظرم همون اعداد استفاده بشه بهتره. نظر شما چیه؟

- در فایل ترجمه شده مشاهده میشه که بعضی خطوط به چند خط تقسیم‌ شده‌اند و کاراکترهای linebreak وجود داره که تو متن اصلی وجود نداره. این رو میشه از روی تعداد خطوط حذف شده و درج شده کل PR هم حدس زد. در زمان نگارش این کامنت ۴۷ خط حذف و ۵۴ خط اضافه شده که یعنی ۷ خط جدید. اولین باره که به این تفاوت توجه میکنم. این‌هارو توی پیشنهادهای توصیه‌شده ام نیوردم تا خود توصیه‌ها مربوط به خود ترجمه باشه. مورد زیاد مهمی نیست. ولی اگه رعایت بشه بهتره تا یه دستی با منبع اصلی از نظر تعداد پاراگراف‌ها حفظ بشه.

- مورد بعدی این‌که در زبان markdown پاراگراف‌ها با قرار دادن یک خط خالی بینشون جدا میشن و ضروریه که حتمن قرار داده بشن تا پارگراف ایجاد بشه. در غیر این صورت به صورت یک پارگراف رندر میشن. یک مورد دیده شد و اصلاحش پیشنهاد داده شد. در این موارد از سایت ایجاد‌شده توسط netlify و مقایسه آن با سایت اصلی هم میشه این مورد رو دید. ولی دونستنش فکر کنم کفایت کنه.

- تو ترجمه کلمه sibling دیدم که یه‌دستی نیست. اصلاحات پیشنهادی‌ام رو بر اساس اولین ترجمه که برادر بود قرار دادم. ولی ترجمه اصلیش هم‌نیا ست. کلمه‌ای نیست که مثل زبان انگلیسی توی مکالمات روزمره استفاده شه و موندم بین درستی ترجمه ویا روانی و قابل فهم بودن ترجمه. خواستم بدونم نظر شما چیه؟

پی‌نوشت: درسته که من درخواست مرور توسط خودم رو ثبت کردم. ولی به دلیل مشکلات کاری نتونستم با شما همراهی کنم. این موارد اتفاق میافته و نظرم اینه که تو این مورد اگر دیدیم بعد یه مدت زمانی review ثبت نشد merge رو انجام بدیم. بدیهیه که اصلاحات رو میشه تو PR های جداگانه رسوند و به بحث گذاشت. مهم اینه که کار پیش بره. ترجیهم اینه که یه کار ۱۰۰٪ انجام شه تا این که چند کار ۸۰٪ داشته باشیم.

راستی این کامنت هم فارسی نوشتم چون دیدم بهتره فارسی بنوسیم تا به کل مشارکت کنندگان فارغ از سطح زبانشون مفهوم یکسانی رو برسونم.

Approved instead of Request changes

sorousht · 2019-04-10T09:21:06Z

@sJJdGG عزیز
با مواردی که گفتید موافقم. در مورد numbers، اون چیزی که من متوجه شدم، چندین جا به عنوان اسم متغییر بوده و نباید ترجمه می‌شده. اگر منظور شما رو درست متوجه نشدم، شماره خط رو به من بگید.
درمورد sibling هم پیشنهاد من معادل "برادر" هست.
راستی ما بعضی از هماهنگی‌ها و مشورت‌ها رو توی اسلک انجام می‌دیم. چرا بعد از گذشت این همه وقت هنوز عضو نشدید؟ اگر مایل هستید، لطفا یک ایمیل به من بزنید.
ممنون

Co-Authored-By: zahrajoulaei <zahrajoulaei@gmail.com>

zahrajoulaei · 2019-04-10T20:39:27Z

@sJJdGG ممنونم از بررسی تون
همون طور که سروش گفتن دلیل ترجمه نکردن number متغیر بودنش بوده.مگر اینکه جایی بوده باشه که اشتباه کردم و حواسم نبوده؟

نمیدونم دلیل ایجاد linebreak ها چی بود.ممنون از دقت تون

برای کلمه ی sibling هم نظر سروش همتا یا برادر بود اما باید به یه کلمه ی یکسان برای همه ی فایل ها برسیم و تو واژه نامه قرارش بدیم

تغییرات ‍پیشهنادی تونو اعمال کردم و ممنونم ازتون.

sJJdGG · 2019-04-12T12:14:32Z

@zahrajoulaei @sorousht

در مورد ترجمه numbers این که در متن انگلیسی یه جور ایهام داره و یکی به معنی اعداده و یه اشاره‌ی دیگش به متغیر به اسم numbers‌ئه. پشنهاد من ترجمه این بخش ها به صورتی شبیه این تا مثل متن اصلی هر دو مورد پوشش داده بشه:

An array of numbers

آرایه‌ای از اعداد (با نام متغیر numbers)

zahrajoulaei · 2019-04-13T21:01:03Z

@zahrajoulaei @sorousht

در مورد ترجمه numbers این که در متن انگلیسی یه جور ایهام داره و یکی به معنی اعداده و یه اشاره‌ی دیگش به متغیر به اسم numbers‌ئه. پشنهاد من ترجمه این بخش ها به صورتی شبیه این تا مثل متن اصلی هر دو مورد پوشش داده بشه:

An array of numbers

آرایه‌ای از اعداد (با نام متغیر numbers)

انجام شد.

sorousht · 2019-04-14T16:33:53Z

@zahrajoulaei @sorousht

در مورد ترجمه numbers این که در متن انگلیسی یه جور ایهام داره و یکی به معنی اعداده و یه اشاره‌ی دیگش به متغیر به اسم numbers‌ئه. پشنهاد من ترجمه این بخش ها به صورتی شبیه این تا مثل متن اصلی هر دو مورد پوشش داده بشه:

An array of numbers

آرایه‌ای از اعداد (با نام متغیر numbers)

@zahrajoulaei @sJJdGG

توی متن که گشتم، چهار مورد دیدم که از این کلمه استفاده شده:

Given the code below, we use the map() function to take an array of numbers and double their values.
Below, we loop through the numbers array using the JavaScript map() function.
This code displays a bullet list of numbers between 1 and 5.
We can refactor the previous example into a component that accepts an array of numbers and outputs a list of elements.

به جز مورد سوم، بقیه موارد کلمه numbers به صورت کد استفاده شده است و ما کد ها رو نباید ترجمه کنیم. مورد سوم هم دقیقا باید ترجمه بشه چون منظورش لیستی از اعداد بوده. من ایهامی ندیدم.

و فکر می کنم جمله take an array of `numbers` and double their values رو میشه اینطور ترجمه کرد: یک آرایه numbers را می‌گیرد و مقادیر آن را دوبرابر می‌کند.

sJJdGG · 2019-04-15T09:53:41Z

@sorousht @zahrajoulaei

منظور من از ترجمه numbers در موارد یک و چهار بود که در این جملات کلمه nubmers در وهله اول نه به عنوان یک اسم خاص (یعنی نام متغیر) بلکه به عنوان یک اسم عام به عنوان جمع number به کار رفته. از کجا مطمئنم که اسم عامه؟ به دلیل این که در هر دو مورد کلمه an رو آورده و همانند فارسی که کلمه "یک" برای کلمات عام استفاده میشه (مانند: یک ماشین) در انگلیسی هم کلمه "an" همین نقش رو ایفا میکنه. مثلن در مورد ۴ اگر به طور فرض کامپوننت ریفکتور شده نام آرگومان و پارامتر رو از numbers به x تغییر بدیم در اصل کار کامپوننت که یک آرایه از اعداد میگیره و یک لیست از المنت هارو برمیگردونه تغییری ایجاد نمیشه و متن هم داره نحوه کار این کامپننت رو توضیج میده. ولی همزمان نگارنده برای کمک به خواننده در فهم و روانی خواندن کد از قصد نام متغیر رو هم در کد numbers گذاشته تا هزینه ذهنی فهم همزمان متن و کد و برقراری ارتباط بین بخش‌های مختلف کد و متن را کاهش بده که تصمیم درستی بوده. در این دو مورد من ترجمه numbers به اعداد رو درست میدونم. ولی میشه همانند متن اصلی سعی کرد هزینه ذهنی رو برای خواننده با ارائه توضیحاتی در پرانتز کاهش داد، مانند: (در نمونه کد زیر آرایه numbers را به عنوان ورودی میگیرد). دقت کنید که پیشنهاد من ترجمه کد که در این مورد فرضن شامل تغییر نام متغیر هم در کد و هم در متن به "اعداد" نیست (چه بسا اگر این کار رو میکردیم خود ترجمه ما دقیقن مشابه متن اصلی بود).

مورد ۳ همونطور که سروش گفت داره از numbers به عنوان جمع کلمه number استفاده میکنه.

مورد ۲ هم که داره به متغیر numbers (اسم خاص) اشاره میکنه و همونطوری که خود ری‌اکت به فارسی ترجمه نمیشه اینجا هم numbers نباید ترجمه بشه.

امیدوارم دلایلم رو با توضیح مورد نیازش داده باشم. اگر باز فکر میکنید توضیح بیشتری نیازه من بی‌خبر نذارید.

sorousht · 2019-04-15T16:46:31Z

@sJJdGG ممنون از توضیح دقیق، حرف حساب جواب نداره :)

zahrajoulaei · 2019-04-16T16:52:12Z

@sJJdGG @sorousht ممنونم از توضیحات جفتتون. تغییرات آخر رو ملاحظه کنید. امیدوارم دیگه آخرین کامیت بوده باشه و اکسپت شه.

sorousht

@zahrajoulaei عزیز، ممنون از تغییراتی که انجام دادید، فقط همین یک مورد کوچک هست. قول می دم (از طرف خودم) آخرین تغییر این صغحه باشه‌ :))

zahrajoulaei added 2 commits March 6, 2019 22:35

🎉 initial commit on translate react

2cda00f

🎉 translate lists and keys page

0104e28

zahrajoulaei requested a review from sorousht March 17, 2019 21:38

tesseralis mentioned this pull request Mar 21, 2019

[OLD] Legacy Persian Translation Progress #1

Closed

44 of 87 tasks

📝 fix dictations and spaces

9ca9d3a

sorousht suggested changes Mar 24, 2019

View reviewed changes

📝 do the requested changes

8c2d107

sorousht suggested changes Mar 26, 2019

View reviewed changes

sJJdGG self-requested a review March 27, 2019 10:49

📝 do the requested changes by Soroush

451eed6

sorousht approved these changes Apr 7, 2019

View reviewed changes

sorousht self-requested a review April 7, 2019 08:45

sorousht suggested changes Apr 7, 2019

View reviewed changes

Comment thread content/docs/lists-and-keys.md Resolved

Comment thread content/docs/lists-and-keys.md Resolved

Comment thread content/docs/lists-and-keys.md Resolved

sorousht suggested changes Apr 7, 2019

View reviewed changes

Comment thread content/docs/lists-and-keys.md Resolved

Comment thread content/docs/lists-and-keys.md Resolved

Comment thread content/docs/lists-and-keys.md Resolved

📝 do the requested changes by Soroush

f8c5b2e

sorousht previously approved these changes Apr 8, 2019

View reviewed changes

sJJdGG previously approved these changes Apr 9, 2019

View reviewed changes

sJJdGG requested review from sJJdGG and sorousht April 9, 2019 23:15

Update content/docs/lists-and-keys.md

fa65966

Co-Authored-By: zahrajoulaei <zahrajoulaei@gmail.com>

zahrajoulaei dismissed sorousht’s stale review via fa65966 April 10, 2019 14:04

sJJdGG and others added 8 commits April 10, 2019 16:04

Update content/docs/lists-and-keys.md

d442e57

Co-Authored-By: zahrajoulaei <zahrajoulaei@gmail.com>

Update content/docs/lists-and-keys.md

7ae7259

Co-Authored-By: zahrajoulaei <zahrajoulaei@gmail.com>

Update content/docs/lists-and-keys.md

d9b417b

Co-Authored-By: zahrajoulaei <zahrajoulaei@gmail.com>

Update content/docs/lists-and-keys.md

2a37482

Co-Authored-By: zahrajoulaei <zahrajoulaei@gmail.com>

Update content/docs/lists-and-keys.md

576e736

Co-Authored-By: zahrajoulaei <zahrajoulaei@gmail.com>

Update content/docs/lists-and-keys.md

034928c

Co-Authored-By: zahrajoulaei <zahrajoulaei@gmail.com>

Update content/docs/lists-and-keys.md

b8ddda9

Co-Authored-By: zahrajoulaei <zahrajoulaei@gmail.com>

Update content/docs/lists-and-keys.md

f8cdb05

Co-Authored-By: zahrajoulaei <zahrajoulaei@gmail.com>

📝 do the requested changes by sJJdGG

583393e

zahrajoulaei closed this Apr 14, 2019

zahrajoulaei reopened this Apr 14, 2019

📝 do the requested changes about numbers.

9424e25

sorousht suggested changes Apr 17, 2019

View reviewed changes

Comment thread content/docs/lists-and-keys.md Resolved

📝 add an enter for paragraph!

322d298

sorousht approved these changes Apr 18, 2019

View reviewed changes

sorousht merged commit 9d1cb70 into master Apr 18, 2019

delete-merged-branch Bot deleted the lists_and_keys branch April 18, 2019 14:12

Conversation

zahrajoulaei commented Mar 17, 2019

Uh oh!

netlify Bot commented Mar 17, 2019 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

sorousht commented Mar 18, 2019

Uh oh!

sorousht commented Mar 18, 2019

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

sorousht commented Mar 24, 2019

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

zahrajoulaei commented Apr 6, 2019

Uh oh!

sorousht left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

sorousht commented Apr 8, 2019

Uh oh!

zahrajoulaei commented Apr 8, 2019

Uh oh!

sJJdGG left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

sorousht commented Apr 10, 2019

Uh oh!

zahrajoulaei commented Apr 10, 2019

Uh oh!

sJJdGG commented Apr 12, 2019

Uh oh!

zahrajoulaei commented Apr 13, 2019

Uh oh!

sorousht commented Apr 14, 2019

Uh oh!

sJJdGG commented Apr 15, 2019 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

sorousht commented Apr 15, 2019

Uh oh!

zahrajoulaei commented Apr 16, 2019

Uh oh!

sorousht left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

netlify Bot commented Mar 17, 2019 •

edited

Loading

sJJdGG commented Apr 15, 2019 •

edited

Loading